Skip to content

Improve fatal worker error message#11381

Merged
SimenB merged 3 commits intojestjs:masterfrom
scitech:call-retries-messaging
May 7, 2021
Merged

Improve fatal worker error message#11381
SimenB merged 3 commits intojestjs:masterfrom
scitech:call-retries-messaging

Conversation

@scitech
Copy link
Copy Markdown
Contributor

@scitech scitech commented May 6, 2021

Summary

"Call retries were exceeded" is a bit obscure, with more information users could have an easier time tracking down bugs.

See also #8769

Test plan

Tests were updated

Copy link
Copy Markdown
Member

@SimenB SimenB left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@SimenB
Copy link
Copy Markdown
Member

SimenB commented May 7, 2021

Prettier is a bit unhappy, could you fix?

@ravigore

This comment has been minimized.

@scitech
Copy link
Copy Markdown
Contributor Author

scitech commented May 7, 2021

Prettier is a bit unhappy, could you fix?

All set now @SimenB. Thank you!

@SimenB SimenB merged commit 705af68 into jestjs:master May 7, 2021
@scitech scitech deleted the call-retries-messaging branch May 7, 2021 21:20
mprinc pushed a commit to mprinc/jest that referenced this pull request May 9, 2021
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 7, 2021

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.
Please note this issue tracker is not a help forum. We recommend using StackOverflow or our discord channel for questions.

@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Jun 7, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants